985203b634ecf2f5ec8ce5f85e34cb16043996cf,processing/src/main/java/io/druid/query/groupby/GroupByQuery.java,GroupByQuery,GroupByQuery,#DataSource#QuerySegmentSpec#DimFilter#QueryGranularity#List#List#List#HavingSpec#LimitSpec#Map#,114

Before Change


    this.limitSpec = (limitSpec == null) ? new NoopLimitSpec() : limitSpec;

    Preconditions.checkNotNull(this.granularity, "Must specify a granularity");
    Queries.verifyAggregations(this.aggregatorSpecs, this.postAggregatorSpecs);

    // Verify no duplicate names between dimensions, aggregators, and postAggregators.
    // They will all end up in the same namespace in the returned Rows and we can't have them clobbering each other.

After Change


      Preconditions.checkArgument(spec != null, "dimensions has null DimensionSpec");
    }
    this.aggregatorSpecs = aggregatorSpecs == null ? ImmutableList.<AggregatorFactory>of() : aggregatorSpecs;
    this.postAggregatorSpecs = Queries.prepareAggregations(
        this.aggregatorSpecs,
        postAggregatorSpecs == null ? ImmutableList.<PostAggregator>of() : postAggregatorSpecs
    );
    this.havingSpec = havingSpec;
    this.limitSpec = (limitSpec == null) ? new NoopLimitSpec() : limitSpec;